Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update logstash docker to use ubuntu 20.04 base image #13442

Merged
merged 3 commits into from
Dec 9, 2021
Merged

Conversation

robbavey
Copy link
Member

@robbavey robbavey commented Nov 23, 2021

This commit updates the base image of the logstash full and oss docker images to use ubuntu 20.04 as the base image.
The ubi8 image remains unchanged

@robbavey
Copy link
Member Author

This passes CI on x86_64 - https://logstash-ci.elastic.co/job/elastic+logstash+master+multijob-docker-acceptance/186/
Fails on aarch64 due to #13444

@roaksoax
Copy link
Contributor

Fixes #13438

Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am getting locale warnings when starting that I don't recall seeing in the centos image.

❯ docker run docker.elastic.co/logstash/logstash-full:8.1.0-SNAPSHOT
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)

qa/docker/spec/full/container_spec.rb Outdated Show resolved Hide resolved
qa/docker/spec/oss/container_spec.rb Outdated Show resolved Hide resolved
Comment on lines 29 to 32
RUN for iter in {1..10}; do \
{% if image_flavor != 'ubi8' -%}
export DEBIAN_FRONTEND=noninteractive && \
{% endif -%}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was tempted to move this export to outside of the run loop, but that would add an extra RUN layer, so let's just keep it as is 👍

@robbavey
Copy link
Member Author

robbavey commented Dec 8, 2021

Fixed setlocale: LC_ALL: cannot change locale issue and added tests.

Tests pass across distributions and architectures:

https://logstash-ci.elastic.co/job/elastic+logstash+master+multijob-docker-acceptance/198/
https://logstash-ci.elastic.co/job/elastic+logstash+master+multijob-aarch64-docker-acceptance/

Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@robbavey robbavey merged commit 11cf6a1 into main Dec 9, 2021
robbavey added a commit to robbavey/logstash that referenced this pull request Dec 9, 2021
…0.04 base image

Backport elastic#13442 to 8.0 branch. Original message:

* Update logstash docker to use ubuntu 20.04 base image
* Correctly set locale for ubuntu docker image
* tiny typo fix: ubunto -> ubuntu

Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
robbavey added a commit that referenced this pull request Dec 11, 2021
…se image (#13490)

Backport #13442 to 8.0 branch. Original message:

* Update logstash docker to use ubuntu 20.04 base image
* Correctly set locale for ubuntu docker image
* tiny typo fix: ubunto -> ubuntu

Co-authored-by: João Duarte <jsvd@users.noreply.github.com>

Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
robbavey added a commit to robbavey/logstash that referenced this pull request Dec 16, 2021
…20.04 base image

Backport elastic#13442 to 7.17 branch. Original message:

* Update logstash docker to use ubuntu 20.04 base image
* Correctly set locale for ubuntu docker image
* tiny typo fix: ubunto -> ubuntu

Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
robbavey added a commit that referenced this pull request Dec 16, 2021
…ase image (#13529)

Backport #13442 to 7.17 branch. Original message:

* Update logstash docker to use ubuntu 20.04 base image
* Correctly set locale for ubuntu docker image
* tiny typo fix: ubunto -> ubuntu

Co-authored-by: João Duarte <jsvd@users.noreply.github.com>

Co-authored-by: João Duarte <jsvd@users.noreply.github.com>
@jsvd jsvd deleted the docker_ubuntu branch January 18, 2022 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants